Skip to content

Conversation

@joshheald
Copy link
Contributor

Description

This PR adds barcode scanning support to the payment successful screen. Scanning a barcode takes you to a new cart.

Steps to reproduce

  1. Launch the app and open POS
  2. Complete a card or cash payment
  3. Scan a barcode
  4. Observe that you're taken to a new cart, and the item you scanned is added to the cart.

The scanning should not work on the email receipt screen, but it will put the barcode numbers in the text field if you scan there, which we can't reasonably prevent.

Screenshots

scanning.on.payment.success.mp4

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@joshheald joshheald added this to the 22.8 milestone Jul 4, 2025
@joshheald joshheald requested a review from staskus July 4, 2025 14:51
@joshheald joshheald added type: task An internally driven task. feature: POS labels Jul 4, 2025
@joshheald joshheald marked this pull request as ready for review July 4, 2025 14:53
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Jul 4, 2025

App Icon📲 You can test the changes from this Pull Request in WooCommerce iOS Prototype by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS Prototype
Build Numberpr15870-35c21f9
Version22.7
Bundle IDcom.automattic.alpha.woocommerce
Commit35c21f9
Installation URL2fihdbj1cp4no
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Copy link
Contributor

@staskus staskus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢 Looks good! Tested on iPad 26.0 device with a barcode scanner after cash and card payments. Verified that it doesn't work on an email receipt screen.

@joshheald joshheald merged commit 08251c7 into trunk Jul 7, 2025
13 checks passed
@joshheald joshheald deleted the woomob-692-woo-posbarcodes-scanning-on-the-order-complete-screen-should branch July 7, 2025 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: POS type: task An internally driven task.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants